Source user object id directly instead of with graph#11531
Conversation
benbp
added
the
Central-EngSys
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the user object ID retrieval logic to source it directly from the Azure context instead of using Azure AD Graph API calls. The change simplifies the code by eliminating multiple fallback attempts to lookup users via email and UserPrincipalName.
- Replaces Graph API user lookups with direct extraction from Azure context ExtendedProperties
- Removes fallback logic for corp tenant and TME tenant user ID lookups
- Simplifies error handling by consolidating multiple null checks
benbp
force-pushed
the
benbp/deploy-no-graph
branch
from
9b92452 to
3b416a2
Compare
|
The following pipelines have been queued for testing: |
benbp
force-pushed
the
benbp/deploy-no-graph
branch
from
3b416a2 to
5827e7a
Compare
|
The following pipelines have been queued for testing: |
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#11531 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
Source user object id directly instead of with graph